Repository navigation
Conversation
This was referenced Oct 28, 2026
This was referenced Oct 28, 2026
yohamta0
force-pushed
the
stack/12-ui07
branch
from
October 29, 2026 09:21
2be6245 to
b4c9717
Compare
yohamta0
force-pushed
the
stack/11-ui06
branch
2 times, most recently
from
October 29, 2026 12:58
731a3b7 to
09e506e
Compare
yohamta0
force-pushed
the
stack/12-ui07
branch
from
October 29, 2026 12:58
b4c9717 to
14bc39e
Compare
yohamta0
force-pushed
the
stack/11-ui06
branch
from
October 30, 2026 06:44
09e506e to
39da1eb
Compare
yohamta0
force-pushed
the
stack/12-ui07
branch
from
October 30, 2026 06:44
14bc39e to
71d9ece
Compare
yohamta0
force-pushed
the
stack/11-ui06
branch
from
October 30, 2026 08:43
39da1eb to
dc398f0
Compare
yohamta0
force-pushed
the
stack/12-ui07
branch
2 times, most recently
from
October 31, 2026 04:50
79e7296 to
f551129
Compare
yohamta0
force-pushed
the
stack/11-ui06
branch
from
October 31, 2026 05:37
6188c7e to
e4ac932
Compare
yohamta0
force-pushed
the
stack/12-ui07
branch
2 times, most recently
from
October 31, 2026 09:21
361ac5e to
4bcacec
Compare
yohamta0
force-pushed
the
stack/11-ui06
branch
2 times, most recently
from
October 31, 2026 10:39
3ddec57 to
ade3afa
Compare
…s, tab strip (dagucloud#3391) * UI: visual-editor/canvas, visual-editor/doc, visual-editor/(root) * Fix formatter, dirty check and duplicate bugs * Guard lookups by user keys and label sheet panel * Fix compare, step format, clipboard and run label
Two steps with the same issue message both pointed at the first step; a row now reveals the step of its own structured issue. Toggling a favourite in the add-step list left the highlight on the row position, so a second Alt+Shift+F toggled another action. The active row now follows the toggled action. An env list of one-key maps with a repeated key sent edits to the wrong item; edits now target the item that gives the value. Form diffs, field issue groups and agent run counts treated keys such as `constructor` as inherited properties; they now use own keys only. A step that cannot be read and has no name could get its placeholder label written into a route. The YAML tokenizer ended a block scalar of a list item at the wrong column, so the next key of the item lost its colour and required-key mark.
useYamlSection set a host state that disagreed with what showed back to the old kind. A link that turned the host 'expanded' (Open step on a failure, a run tab) while the YAML file showed, or while the section was closed, was rewritten to 'docked', so the file opened instead of the step. A host that turns 'expanded' now decides what shows; a 'docked' link for a shown step is still set right. An issue row was a button whenever it had a step or a line, even when no callback could act on it: a row with only a line and no onRevealIssue, or a value reference with only onRevealIssue, closed the popover and did nothing. A row is a button now only when a callback acts on it. readEnv wrote a `__proto__=...` entry into a plain object, which dropped it, so the env editor lost it on the next edit. The entries have no prototype now. The popover tests use role queries in place of test ids.
The + button of an edge inferred from a step reference inserts a step between the two ends, and the ghost shows it there. The edit only made the new step depend on the source. The target lists no depends, so it kept running beside the new step instead of after it. The new step now joins the depends of a target that does not list the source. The reference of the target still resolves: the source is upstream of the new step.
yohamta0
force-pushed
the
stack/12-ui07
branch
from
October 31, 2026 10:39
4bcacec to
df34b0d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All PRs of the stack
Summary
This PR adds the graph model of the canvas (graph, edge routing, gestures, ghost node, palette model), the Add step panel, the issues popover with change bars, and the run bar with draft recovery. It also adds a YAML tokenizer and form diff helpers. It adds 42 new files and changes no existing file.
Why
The canvas needs a model of nodes and edges before it can draw them. The editor also needs a way to add steps, to show issues, to mark changed lines and to keep a draft. This PR follows PR 11 (edit session, format, run actions). PR 13 draws the edges and layout on this model, and PR 15 adds the node components.
Changes
visual-editor/canvas):graph.ts(resolve step names and ids),edgeRouting.ts,gestures.ts(connections drawn by the user),ghost.tsandedges/GhostEdge.tsx,edges/useEdgeCurve.ts,edges/dash.ts,paletteModel.ts,agentHeader.ts,ContextMenu.tsx.AddStepPanel.tsx,addStepAliases.ts,YamlSnippet.tsx,yamlTokenizer.ts(coloured pieces of a line).dag-editor):IssuesPopover.tsx,lineChanges.ts,useChangeBars.ts,useYamlSection.ts;inspector/FieldIssues.tsx,inspector/issues.ts.inspector/formDiff.ts,inspector/sections/flow/envForm.ts.visual-editor/run):RunBar.tsx,useDraftRecovery.ts,draftStorage.ts,useRunSpecStale.ts,agentRun.ts.How to review
canvas/graph.tsandcanvas/edgeRouting.ts.canvas/gestures.tsandcanvas/ghost.ts.AddStepPanel.tsx.run/useDraftRecovery.tsandrun/RunBar.tsx.IssuesPopover.tsx,lineChanges.ts,useChangeBars.ts.Most attention:
useDraftRecovery.ts(when a draft is kept or dropped), and the line matching inlineChanges.ts.Tests
AddStepPanel.test.tsx,YamlSnippet.test.tsx,yamlTokenizer.test.ts.dag-editor/__tests__/IssuesPopover.test.tsx,lineChanges.test.ts,useChangeBars.test.tsx.run/__tests__/RunBar.test.tsx,useDraftRecovery.test.ts,draftStorage.test.ts,useRunSpecStale.test.tsx,agentRun.test.ts.canvas/__tests__/agentHeader.test.ts;inspector/__tests__/envForm.test.ts,formDiff.test.ts.step-details/__tests__/PanelShell.test.tsx.Screenshots
These components are used by later PRs; the screenshots show the YAML panel and the run bar in the final UI.